Skip to content

Add missing #include <boost/container_hash/hash.hpp> in tree.cc#40

Closed
DevilishSpirits wants to merge 1 commit into
google:mainfrom
DevilishSpirits:main
Closed

Add missing #include <boost/container_hash/hash.hpp> in tree.cc#40
DevilishSpirits wants to merge 1 commit into
google:mainfrom
DevilishSpirits:main

Conversation

@DevilishSpirits

Copy link
Copy Markdown

The title say everything.

This is a tiny fix to make "it works on my system™" which is an ArchLinux with boost package version 1.91.0-1.

@google-cla

google-cla Bot commented Jun 3, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@DevilishSpirits

Copy link
Copy Markdown
Author

I signed the CLA.

@fdegros

fdegros commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Is it essentially the same fix as 8b43dcf ? Meaning, if you use 8b43dcf instead of this pull request, does it solve the build issue on your system?

@DevilishSpirits

Copy link
Copy Markdown
Author

Both fixes are not identical but equivalent. On my system <boost/functional/hash.hpp> is :

// Copyright 2005-2009 Daniel James.
// Distributed under the Boost Software License, Version 1.0. (See accompanying
// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)

#include <boost/container_hash/hash.hpp>

And yes, it solves my build issues 👍.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants